Skip to content

region 3 4 complete tests and lockouts rework #23720

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Aug 27, 2025
Merged

region 3 4 complete tests and lockouts rework #23720

merged 2 commits into from
Aug 27, 2025

Conversation

@michalek-no
Copy link
Contributor

@michalek-no michalek-no commented Aug 5, 2025

No description provided.

@michalek-no michalek-no requested a review from a team as a code owner August 5, 2025 11:45
@michalek-no michalek-no requested a review from a team August 5, 2025 11:45
@michalek-no michalek-no added the DNM label Aug 5, 2025
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 5, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 5, 2025 •
edited
Loading

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 24

Inputs:

Sources:

sdk-nrf: PR head: 48d91fe528e677d6f1bb573bba851fc91d58c6b6
mcuboot: PR head: 285fd59f4386a0317e476da5484f67b906073296

more details

sdk-nrf:

PR head: 48d91fe528e677d6f1bb573bba851fc91d58c6b6
merge base: 693f67dc9b63261e2c87ff770263f503ae8a9668
target head (main): 693f67dc9b63261e2c87ff770263f503ae8a9668
Diff

mcuboot:

PR head: 285fd59f4386a0317e476da5484f67b906073296
merge base: ecc13ac1b0336c5ff39fc00075d34b55ab619752
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (13) 
bootloader
│  ├── mcuboot
│  │  ├── boot
│  │  │  ├── zephyr
│  │  │  │  ├── Kconfig
│  │  │  │  │ main.c
subsys
│  ├── bootloader
│  │  ├── Kconfig
│  │  ├── bl_boot
│  │  │  │ bl_boot.c
tests
│  ├── subsys
│  │  ├── bootloader
│  │  │  ├── b0_lock
│  │  │  │  │ testcase.yaml
│  │  │  ├── b0_lock_rwx
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  ├── sysbuild.conf
│  │  │  │  ├── sysbuild
│  │  │  │  │  ├── mcuboot
│  │  │  │  │  │  │ prj.conf
│  │  │  │  │ testcase.yaml
west.yml

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
    • ◻️ test-fw-nrfconnect-chip
    • ◻️ test-fw-nrfconnect-tfm
    • ◻️ test-sdk-find-my
    • ◻️ test-sdk-mcuboot
    • ◻️ test-sdk-dfu
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@github-actions
Copy link

github-actions bot commented Aug 5, 2025

You can find the documentation preview for this PR here.

@michalek-no michalek-no changed the title wip region 3 4 complete tests and lockouts rework DRAFT Aug 8, 2025
@michalek-no michalek-no force-pushed the mb-next-lock branch 4 times, most recently from acb4612 to 8202dc8 Compare August 11, 2025 05:54
@michalek-no michalek-no requested review from a team as code owners August 11, 2025 05:54
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 11, 2025 •
edited by github-actions bot
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
mcuboot nrfconnect/sdk-mcuboot@ecc13ac nrfconnect/sdk-mcuboot@285fd59 (main) nrfconnect/[email protected]

✅ All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@michalek-no michalek-no force-pushed the mb-next-lock branch 5 times, most recently from cc5d4af to 4280ea8 Compare August 13, 2025 05:21
@michalek-no
Copy link
Contributor Author

michalek-no commented Aug 13, 2025

rebase

@michalek-no michalek-no changed the title region 3 4 complete tests and lockouts rework DRAFT region 3 4 complete tests and lockouts rework Aug 13, 2025
@michalek-no michalek-no removed the DNM label Aug 13, 2025
nordicjm
nordicjm requested changes Aug 20, 2025
View reviewed changes
@michalek-no michalek-no requested a review from nordicjm August 20, 2025 10:22
@michalek-no michalek-no force-pushed the mb-next-lock branch 2 times, most recently from 9e394f2 to 8e62a6c Compare August 21, 2025 12:40
@michalek-no
Copy link
Contributor Author

michalek-no commented Aug 21, 2025

rebase

@michalek-no
Copy link
Contributor Author

michalek-no commented Aug 26, 2025

rebase

@michalek-no
manifest: update sdk-mcuboot
d1856ab 
pull-in self lock RWX feature.

Signed-off-by: Mateusz Michalek <[email protected]>
@michalek-no
tests: bootloader: NSIB MCUBoot locks
48d91fe 
complete set of tests for region 3 and 4 lockouts.

Signed-off-by: Mateusz Michalek <[email protected]>
@github-actions github-actions bot removed the DNM label Aug 27, 2025
@michalek-no michalek-no added DNM and removed DNM labels Aug 27, 2025
@nordicjm nordicjm merged commit cc5f4ee into nrfconnect:main Aug 27, 2025
15 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nvlsianpu nvlsianpu nvlsianpu approved these changes

@nordicjm nordicjm nordicjm approved these changes

@fundakol fundakol Awaiting requested review from fundakol

@de-nordic de-nordic Awaiting requested review from de-nordic

@gchwier gchwier Awaiting requested review from gchwier

Assignees

No one assigned

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-mcuboot

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@michalek-no @NordicBuilder @nvlsianpu @nordicjm